Skip to content

Conversation

@politikl
Copy link
Contributor

@politikl politikl commented Oct 27, 2025

Summary

This PR reorganizes rustdoc tests into their correct subdirectories for better categorization and maintainability.

Changes

  • Moved 6 lint-related tests to tests/rustdoc-ui/lints/
  • Moved 2 intra-doc link tests to tests/rustdoc-ui/intra-doc/
  • Moved 3 deref-related tests to tests/rustdoc-ui/deref/
  • Moved 1 doc-cfg test to tests/rustdoc/doc-cfg/

Tests

All moved tests have been verified and pass in their new locations.

Reorganize rustdoc tests into their correct subdirectories for better
categorization:
- Move lint-related tests to rustdoc-ui/lints/
- Move intra-doc link tests to rustdoc-ui/intra-doc/
- Move deref-related tests to rustdoc-ui/deref/
- Move doc-cfg test to rustdoc/doc-cfg/

This improves test organization and makes it easier to find tests
related to specific rustdoc functionality.
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Oct 27, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 27, 2025

r? @GuillaumeGomez

rustbot has assigned @GuillaumeGomez.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@apiraino
Copy link
Contributor

I struggle a bit to see the connection between "fixes #136452" (a pull request, not a ticket) and this patch. I don't even see there a comment or anything resembling a TODO item.

@GuillaumeGomez
Copy link
Member

The "fixes" comment is wrong but the cleanup is good though. Did you link to the wrong #?

@politikl
Copy link
Contributor Author

politikl commented Oct 29, 2025

You are correct, I did put the wrong number on the pr! I accidentally added the "Fixes #136452" and it was a general cleanup job. I just removed it from the comment.

@GuillaumeGomez
Copy link
Member

Thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 29, 2025

📌 Commit 0715745 has been approved by GuillaumeGomez

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 29, 2025
bors added a commit that referenced this pull request Oct 29, 2025
Rollup of 4 pull requests

Successful merges:

 - #148159 (Move rustdoc tests to appropriate subdirectories)
 - #148218 (Update T-compiler/ops Zulip messages)
 - #148228 (Run regression test for #147964 on next solver too)
 - #148237 (citool: Always print a signed percentage in job duration changes)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 78791eb into rust-lang:master Oct 29, 2025
11 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Oct 29, 2025
rust-timer added a commit that referenced this pull request Oct 29, 2025
Rollup merge of #148159 - politikl:move-rustdoc-tests-to-subdirs, r=GuillaumeGomez

Move rustdoc tests to appropriate subdirectories

## Summary

This PR reorganizes rustdoc tests into their correct subdirectories for better categorization and maintainability.

## Changes

- Moved 6 lint-related tests to `tests/rustdoc-ui/lints/`
- Moved 2 intra-doc link tests to `tests/rustdoc-ui/intra-doc/`
- Moved 3 deref-related tests to `tests/rustdoc-ui/deref/`
- Moved 1 doc-cfg test to `tests/rustdoc/doc-cfg/`

## Tests

All moved tests have been verified and pass in their new locations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants